Skip to content

Reposition guide within help widget#4271

Merged
walterbender merged 1 commit intosugarlabs:masterfrom
Ubayed-Bin-Sufian:mb_guide_link_reposition
Jan 14, 2025
Merged

Reposition guide within help widget#4271
walterbender merged 1 commit intosugarlabs:masterfrom
Ubayed-Bin-Sufian:mb_guide_link_reposition

Conversation

@Ubayed-Bin-Sufian
Copy link
Contributor

@Ubayed-Bin-Sufian Ubayed-Bin-Sufian commented Jan 14, 2025

Summary

This pull request repositions the guide link within the help widget to enhance usability for students by directing them to the guide.html file for study purposes.

Changes introduced

  • Moved the guide link in the help widget to the 3rd position (immediately after the 'Meet Mr. Mouse' section). Previously, it was located at the end of the menu.
  • Improved code readability by rearranging certain sections.

Screenshots/video

Screenshare.-.2025-01-14.3_07_18.PM.mp4

Checklist

  • The changes adhere to the project's contribution guidelines.
  • Code changes have been implemented and tested.

@Ubayed-Bin-Sufian
Copy link
Contributor Author

@walterbender & @pikurasa. Kindly review it.

@walterbender
Copy link
Member

I think this is a good change. However, you have included many unrelated changes to this PR as well, including some string changes that impact translation. Please make these changes in a separate PR.

js/turtledefs.js Outdated
"myblocks"
],
["graphics", "pen", "media", "sensors", "ensemble"]
[
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

unintended

js/turtledefs.js Outdated
_("Mr Mouse is our Music Blocks conductor.") +
" " +
_("Mr Mouse encourages you to explore Music Blocks.") +
_("Mr Mouse encourages you to explore Music Blocks.") +
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

uninteneded

js/turtledefs.js Outdated
_("Mr Mouse encourages you to explore Music Blocks.") +
" " +
_("Let us start our tour!"),
_("Let us start our tour!"),
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

uninteneded

js/turtledefs.js Outdated
window.btoa(
base64Encode(SCROLLUNLOCKBUTTON))

window.btoa(base64Encode(SCROLLUNLOCKBUTTON))
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

uninteneded

js/turtledefs.js Outdated

window.btoa(base64Encode(SCROLLUNLOCKBUTTON))
]);
// TODO: JavaScript Editor
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

? how

js/turtledefs.js Outdated
window.btoa(
base64Encode(WRAPTURTLEBUTTON))

window.btoa(base64Encode(WRAPTURTLEBUTTON))
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

uninteneded

js/turtledefs.js Outdated
window.btoa(
base64Encode(RESTORETRASHBUTTON))

window.btoa(base64Encode(RESTORETRASHBUTTON))
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

uninteneded

@Ubayed-Bin-Sufian Ubayed-Bin-Sufian force-pushed the mb_guide_link_reposition branch 2 times, most recently from edda5c4 to 7cc73af Compare January 14, 2025 19:38
@Ubayed-Bin-Sufian
Copy link
Contributor Author

I think this is a good change. However, you have included many unrelated changes to this PR as well, including some string changes that impact translation. Please make these changes in a separate PR.

@walterbender I have removed the unrelated changes. Please review it.

@walterbender walterbender merged commit 23206c9 into sugarlabs:master Jan 14, 2025
4 checks passed
@Ubayed-Bin-Sufian Ubayed-Bin-Sufian deleted the mb_guide_link_reposition branch February 12, 2025 15:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants